Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update the hardcoded DEFAULT_ASSUME_VALID_TARGET #4525

Merged
merged 1 commit into from
Jul 18, 2024
Merged

Update the hardcoded DEFAULT_ASSUME_VALID_TARGET #4525

merged 1 commit into from
Jul 18, 2024

Conversation

eval-exec
Copy link
Collaborator

@eval-exec eval-exec commented Jul 17, 2024 •
edited
Loading

What problem does this PR solve?

We haven't update the hardcoded DEFAULT_ASSUME_VALID_TARGET since Apr 22, update it now.

What's Changed:

Related changes

  • Update the DEFAULT_ASSUME_VALID_TARGET

Check List

Tests

  • No code ci-runs-only: [ quick_checks,linters ]

Side effects

  • None

Release note 

Title Only: Include only the PR title in the release note.

@eval-exec eval-exec requested a review from a team as a code owner July 17, 2024 06:01
@eval-exec eval-exec requested review from quake and removed request for a team July 17, 2024 06:01
@quake quake added this pull request to the merge queue Jul 18, 2024
Merged via the queue into nervosnetwork:develop with commit fc853d2 Jul 18, 2024
33 checks passed
@eval-exec eval-exec mentioned this pull request Jul 19, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@quake quake quake approved these changes

@driftluo driftluo driftluo approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@eval-exec @quake @driftluo